-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Normalize URLs in HMAC interceptor #114
Conversation
- and unit tests thereof - also, uncomment the HMAC debugging code (if variable GP_VERBOSE is true) Fixes: IBM-Cloud#113
GaasHmac.prototype.SEP = "\n"; | ||
GaasHmac.prototype.ENC = "ascii"; // ISO-8859-1 not supported! | ||
GaasHmac.prototype.HMAC_SHA1_ALGORITHM = 'sha1'; // "HmacSHA1"; | ||
GaasHmac.prototype.forceDate = null; | ||
GaasHmac.prototype.forceDateString = null; | ||
|
||
// GaasHmac.prototype.VERBOSE = process.env.GP_VERBOSE || false; | ||
GaasHmac.prototype.VERBOSE = process.env.GP_VERBOSE || false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was mostly disabled because of the lint rule, but I fixed the lint rule.
@@ -36,13 +38,14 @@ GaasHmac.prototype.user = undefined; | |||
GaasHmac.prototype.secret = undefined; | |||
|
|||
GaasHmac.prototype.AUTH_SCHEME = "GaaS-HMAC"; | |||
// GaasHmac.prototype.AUTH_SCHEME = "GP-HMAC"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should switch the scheme in the future?
Note this is to work around swagger-api/swagger-js#1220 — could remove the around once fixed. |
Fixes: #113